From ebb436022534e7ed88adea3e9b0bb0c0bb239e74 Mon Sep 17 00:00:00 2001 From: Chad Horohoe Date: Mon, 22 Feb 2010 00:54:39 +0000 Subject: [PATCH] Cleanup r62798: If you add a new param then document it, simplify check for --force --- maintenance/deleteArchivedFiles.php | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/maintenance/deleteArchivedFiles.php b/maintenance/deleteArchivedFiles.php index 3d8253fc9f..f5d1f27dba 100644 --- a/maintenance/deleteArchivedFiles.php +++ b/maintenance/deleteArchivedFiles.php @@ -30,6 +30,7 @@ class DeleteArchivedFiles extends Maintenance { parent::__construct(); $this->mDescription = "Deletes all archived images."; $this->addOption( 'delete', 'Perform the deletion' ); + $this->addOption( 'force', 'Force deletion of rows from filearchive' ); } public function execute() { @@ -37,10 +38,7 @@ class DeleteArchivedFiles extends Maintenance { $this->output( "Use --delete to actually confirm this script\n" ); return; } - $force = false; - if( $this->hasOption('force') ) { - $force = true; - } + $force = $this->hasOption( 'force '); # Data should come off the master, wrapped in a transaction $dbw = wfGetDB( DB_MASTER ); $dbw->begin(); -- 2.20.1